Skip to content

[WIP] try to support v0.24#221

Closed
SimonDanisch wants to merge 1 commit intomasterfrom
sd/v0.24
Closed

[WIP] try to support v0.24#221
SimonDanisch wants to merge 1 commit intomasterfrom
sd/v0.24

Conversation

@SimonDanisch
Copy link
Copy Markdown
Member

Sadly, GraphMakie seems to be particularly hit by the deprecations, so this will probably take a bit of time.
Could be cleaner and better, to completely switch to using the compute graph, but harder for me since I don't know the recipe that well.

@dgleich
Copy link
Copy Markdown

dgleich commented Jun 26, 2025

@SimonDanisch are there any good sources of Makie 0.23 -> 0.24 conversions? I saw someone mention looking at basic_recipes to see what these look like, but then MakieOrg/Makie.jl#5114 seems to suggest that many of these aren't done. If we can get a set of these, then some of the AI tools may be able to do much of these updates automatically. Other notes I found were https://docs.makie.org/dev/explanations/compute-pipeline#ComputeGraph

@SimonDanisch
Copy link
Copy Markdown
Member Author

We have an upgrade guide in the blogpost:
https://makie.org/website/blogposts/v0.24.0/

@dgleich
Copy link
Copy Markdown

dgleich commented Jun 26, 2025

Thanks. Any other notes / ports / etc. are helpful to give the AI stuff guidance. With small bits of help, Claude was able to muddle it's way to getting a plot to show. The tests aren't passing yet, and I think there are some subtle bugs in what it's done.

@SimonDanisch
Copy link
Copy Markdown
Member Author

Can you push it? or PR it?

@dgleich
Copy link
Copy Markdown

dgleich commented Jun 26, 2025

I'll try and set this up later tonight.

@dgleich
Copy link
Copy Markdown

dgleich commented Jun 27, 2025

Here's the one with more edits along with some quick notes on what's working/ what isn't.
#222

@hexaeder
Copy link
Copy Markdown
Collaborator

hexaeder commented Jul 7, 2025

Thanks for starting on that, finished on #223

@hexaeder hexaeder closed this Jul 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants